Skip to content

Update FillTrigger.cxx#577

Merged
kjplows merged 7 commits into
developfrom
bugfix/trigger_uint
Sep 30, 2025
Merged

Update FillTrigger.cxx#577
kjplows merged 7 commits into
developfrom
bugfix/trigger_uint

Conversation

@nathanielerowe

Copy link
Copy Markdown
Contributor

The CI was picking up some differences between trigger objects caused by recent SBND specific changes. This should resolve one of the issues.

Depends on: SBNSoftware/sbnanaobj#158

@nathanielerowe nathanielerowe added the bugfix Addresses one or more bugs label Sep 17, 2025
@nathanielerowe

Copy link
Copy Markdown
Contributor Author

trigger build ci_ref=v10_09_00 LArSoft/lar*@LARSOFT_SUITE_v10_10_02 SBNSoftware/sbndaq_artdaq_core@v1_10_06 SBNSoftware/sbnd_data@v01_36_00 SBNSoftware/sbnanaobj#158

@kjplows
kjplows self-requested a review September 17, 2025 20:04

@kjplows kjplows left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

there's an implicit cast here - see https://github.com/SBNSoftware/sbnanaobj/blob/develop/sbnanaobj/StandardRecord/SRTrigger.h#L16

Is this change necessary for CI?

@nathanielerowe

Copy link
Copy Markdown
Contributor Author

trigger build ci_ref=v10_09_00 LArSoft/lar*@LARSOFT_SUITE_v10_10_02 SBNSoftware/sbndaq_artdaq_core@v1_10_06 SBNSoftware/sbnd_data@v01_36_00 SBNSoftware/sbnanaobj#158

@nathanielerowe

Copy link
Copy Markdown
Contributor Author

trigger build ci_ref=v10_09_00 LArSoft/lar*@LARSOFT_SUITE_v10_10_02 SBNSoftware/sbndaq_artdaq_core@v1_10_06 SBNSoftware/sbnd_data@v01_36_00 SBNSoftware/sbnanaobj#158

@nathanielerowe

Copy link
Copy Markdown
Contributor Author

trigger build ci_ref=v10_09_00 LArSoft/lar*@LARSOFT_SUITE_v10_10_02 SBNSoftware/sbndaq_artdaq_core@v1_10_06 SBNSoftware/sbnd_data@v01_36_00 SBNSoftware/sbnanaobj#158

@nathanielerowe

Copy link
Copy Markdown
Contributor Author

trigger build ci_ref=v10_09_00 LArSoft/lar*@LARSOFT_SUITE_v10_10_02 SBNSoftware/sbndaq_artdaq_core@v1_10_06 SBNSoftware/sbnd_data@v01_36_00 SBNSoftware/sbnanaobj#158

@SBNSoftware SBNSoftware deleted a comment from FNALbuild Sep 18, 2025
@SBNSoftware SBNSoftware deleted a comment from FNALbuild Sep 18, 2025
@SBNSoftware SBNSoftware deleted a comment from FNALbuild Sep 18, 2025
@SBNSoftware SBNSoftware deleted a comment from FNALbuild Sep 18, 2025
@SBNSoftware SBNSoftware deleted a comment from FNALbuild Sep 18, 2025
@SBNSoftware SBNSoftware deleted a comment from FNALbuild Sep 18, 2025
@SBNSoftware SBNSoftware deleted a comment from FNALbuild Sep 18, 2025
@SBNSoftware SBNSoftware deleted a comment from FNALbuild Sep 18, 2025
@SBNSoftware SBNSoftware deleted a comment from FNALbuild Sep 18, 2025
@SBNSoftware SBNSoftware deleted a comment from FNALbuild Sep 18, 2025
@SBNSoftware SBNSoftware deleted a comment from FNALbuild Sep 18, 2025
@SBNSoftware SBNSoftware deleted a comment from FNALbuild Sep 18, 2025
@SBNSoftware SBNSoftware deleted a comment from FNALbuild Sep 18, 2025
@SBNSoftware SBNSoftware deleted a comment from FNALbuild Sep 18, 2025
@SBNSoftware SBNSoftware deleted a comment from FNALbuild Sep 18, 2025
@SBNSoftware SBNSoftware deleted a comment from FNALbuild Sep 18, 2025
@SBNSoftware SBNSoftware deleted a comment from FNALbuild Sep 18, 2025
@SBNSoftware SBNSoftware deleted a comment from FNALbuild Sep 18, 2025
@SBNSoftware SBNSoftware deleted a comment from FNALbuild Sep 18, 2025
@FNALbuild

Copy link
Copy Markdown

✔️ CI build for LArSoft Succeeded on slf7 for c14:prof -- details available through the CI dashboard

@FNALbuild

Copy link
Copy Markdown

✔️ CI build for LArSoft Succeeded on slf7 for e26:prof -- details available through the CI dashboard

@FNALbuild

Copy link
Copy Markdown

❌ CI build for ICARUS Failed at phase build ICARUS on slf7 for c14:prof -- details available through the CI dashboard

🚨 For more details about the failed phase, check the build ICARUS phase logs

parent CI build details are available through the CI dashboard

@FNALbuild

Copy link
Copy Markdown

❌ CI build for SBND Failed at phase build SBND on slf7 for c14:prof -- details available through the CI dashboard

🚨 For more details about the failed phase, check the build SBND phase logs

parent CI build details are available through the CI dashboard

@FNALbuild

Copy link
Copy Markdown

❌ CI build for ICARUS Failed at phase ci_tests ICARUS on slf7 for e26:prof - ignored warnings for build -- details available through the CI dashboard

🚨 For more details about the failed phase, check the ci_tests ICARUS phase logs

parent CI build details are available through the CI dashboard

@FNALbuild

Copy link
Copy Markdown

⚠️ CI build for SBND Warning at phase ci_tests SBND on slf7 for e26:prof - ignored warnings for build -- details available through the CI dashboard

🚨 For more details about the warning phase, check the ci_tests SBND phase logs

parent CI build details are available through the CI dashboard

@linyan-w linyan-w moved this to Reviewer Approved in SBND 2025 Gen2 Production Sep 18, 2025
@linyan-w linyan-w moved this from Reviewer Approved to Waiting on Reviewer in SBND 2025 Gen2 Production Sep 18, 2025
@nathanielerowe nathanielerowe moved this from Waiting on Reviewer to Wait for next release (for data) in SBND 2025 Gen2 Production Sep 18, 2025
@linyan-w linyan-w moved this from Wait for validation II to Wait for next validation I patch (for MC) in SBND 2025 Gen2 Production Sep 19, 2025
@linyan-w linyan-w moved this from Wait for next validation I patch (for MC) to Wait for validation II in SBND 2025 Gen2 Production Sep 23, 2025
@kjplows kjplows moved this to To merge in SBN software development Sep 29, 2025
@kjplows
kjplows merged commit 92596c4 into develop Sep 30, 2025
@github-project-automation github-project-automation Bot moved this from To merge to Done in SBN software development Sep 30, 2025
@kjplows kjplows moved this from Done to Recently done in SBN software development Sep 30, 2025
@kjplows kjplows moved this from Recently done to Done in SBN software development Oct 1, 2025
@linyan-w linyan-w moved this from Wait for validation II to In tagged release in SBND 2025 Gen2 Production Oct 7, 2025
@nathanielerowe
nathanielerowe deleted the bugfix/trigger_uint branch October 22, 2025 03:42
@kjplows kjplows moved this from Done to 2025 PRs in SBN software development Jan 16, 2026
@kjplows kjplows added this to the v10_10_03_02 milestone Jan 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bugfix Addresses one or more bugs

Projects

Status: Done
Status: 2025 PRs
Status: In tagged release

Development

Successfully merging this pull request may close these issues.

4 participants